Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Relax us ticker frequency requirement. #1123

Merged
merged 3 commits into from
Aug 15, 2019

Conversation

mprse
Copy link
Contributor

@mprse mprse commented Aug 5, 2019

This change is required by the Samsung S111(S5JS100). On this board timer clock used for us ticker operates at 26MHz.
According to current requirements, 8 MHz is the top limit for us ticker timer.

This change relaxes top limit to 100 MHz, but only for 32-bit timers.

This change is required by the Samsung S111(S5JS100). On this board timer clock used for us ticker operates at 26MHz.
According to current requirements, 8 MHz is the top limit for us ticker timer.

This change relaxes top limit to 100 MHz, but only for 32-bit timers.
Amanda Butler added 2 commits August 6, 2019 08:47
Edit file for formatting and precise language.
Copy link
Contributor

@AnotherButler AnotherButler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR 👍

@AnotherButler
Copy link
Contributor

Who should review this from engineering? @jamesbeyond ?

@AnotherButler
Copy link
Contributor

@0xc0170 Who should review this from engineering?

@AnotherButler
Copy link
Contributor

ping @0xc0170

@AnotherButler
Copy link
Contributor

ping @jamesbeyond @0xc0170

Copy link
Contributor

@jamesbeyond jamesbeyond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank alot, this look good

@AnotherButler AnotherButler merged commit 91a3bcf into ARMmbed:development Aug 15, 2019
AnotherButler pushed a commit that referenced this pull request Aug 15, 2019
Apply changes from PR #1123 to v5.13.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants